[6.1] Fix none/codemirror editor width and height by RickR2H · Pull Request #47305 · joomla/joomla-cms

RickR2H · 2026-03-05T10:56:50Z

Pull Request resolves #47292 .

This fixes a width and height bug introduced in PR #46438 when there is no editor loaded or editor is set to CodeMirror .

Summary of Changes

PR #46438 fixed the width and height parameters for TinyMCE. The issue now is that when no editor (editor set to 'none') or CodeMirror is used, the editor collapses because no default width or height is provided.

This occurs, for example, when editing email templates.

Testing Instructions

Testing instructions are in the original PR #46438.

Additional Testing Instructions

Go to System → Mail Templates. In the Options (top right of the screen), set Mail Format to Both.
Open a mail template and check whether the first editor field (where no editor is set) is full width and has a reasonable working height. Also check whether the TinyMCE editor below respects the width and height settings configured in the TinyMCE plugin.
In addition to modifying the modules/mod_custom/mod_custom.xml file, add the code below to the XML after the background image field to have all editor options set in the XML file.

<field
	name="editor1"
	type="editor"
	label="Editor set to CodeMirror"
	height="600"
	width="600"
	editor="codemirror"
/>
<field
	name="editor2"
	type="editor"
	label="Editor set to None"
	height="300"
	width="800"
	editor="none"
/>
<field
	name="editor3"
	type="editor"
	label="Editor set to TinyMCE"
	height="900"
	width="800"
	editor="tinymce"
/>
<field
	name="editor4"
	type="editor"
	label="Use default editor"
/>

Switch the default editor in the configuration to TinyMCE, CodeMirror or None, and check if all editor fields are working accordingly in all places in the site.
Check if the editor is respecting the TinyMCE plugin settings in the frontend of the site.

NOTE:

When upgrading from a previous installation, the editor width and height may still be set to the old default of 750px, which can cause the editor to appear too small. With PR #47326, the default editor dimensions are updated to width 100% and height 550px to ensure the editor displays at the correct size after the upgrade.
When testing this PR make sure to set the width to 100% and the height to 550px in the TinyMCE plugin.

Actual result BEFORE applying this Pull Request

The 'None' editor and CodeMirror editor is collapsed.

Expected result AFTER applying this Pull Request

The 'None' editor and CodeMirror editor is full width and has a usable working height.

Link to documentations

Please select:

Documentation link for guide.joomla.org:
No documentation changes for guide.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed

brianteeman · 2026-03-05T11:34:47Z

46438 silently changed the width of tinymce on NEW installs only with no documentation telling existing users why their editor width is now wrong. Mail templates is just an easy visual indicator of this break. this pr just ignores that. the original pr was simply wrong and all you are doing now is trying to put a band aid on the error. and tomorrow you will need to put a band aid on another area that was broken by the fauly approach in 46438

libraries/src/Form/Field/EditorField.php

RickR2H · 2026-03-05T13:20:25Z

@brianteeman Simply saying that something is wrong is not very constructive.

Maybe a solution would be to check in the individual plugins for the width and height values, as @Fedik suggested. If they are empty us a fallback for the width an height. Could that be a possible solution?

Otherwise I could ask the maintainers to revert my PR. In that case we would be left with an editor that, in my opinion, is not high enough to be practical, and with width and height plugin settings that effectively do nothing. That would also mean users are essentially forced to rely on an external editor plugin by default.

brianteeman · 2026-03-05T13:24:39Z

I am not saying that there is not an issue that needs to be addressed. I am saying that the changes that were made introduced new issues and blaming the user (as I was blamed) for not knowing about an undocumented change that they needed to make is not a suitable solution nor is it a solution to hard code band aid fixes as in this pr.

…t default editor width 750px to 100% for installations upgraded from older versions.

…fix-none-editor

plugins/editors/codemirror/layouts/editors/codemirror/codemirror.php

plugins/editors/tinymce/src/PluginTraits/DisplayTrait.php

brianteeman · 2026-03-06T11:23:47Z

thank you @HLeithner

brianteeman · 2026-03-06T22:58:19Z

FYI the same problem can be seen when editing content in the front end

RickR2H · 2026-03-07T08:21:48Z

Maybe we should remove the width parameter in total and show the editor always 100%. That has been the case in the past. And then just keep the height.

Other option is to remove the current parameters and introduce two new ones.

Any other suggestions?

As this is now a blocker we need a solution quick...

brianteeman · 2026-03-07T08:51:10Z

revert the pr that broke everything and then spend the time to write a complete solution that works correctly. The behaviour this was "fixing" is not new so its not desperate to fix that. Better to get it correct now instead of constantly adding band aid fixes on top

Fedik · 2026-03-07T09:03:17Z

Any other suggestions?

I think it is good as it is, as in the PR.

brianteeman · 2026-03-07T09:27:09Z

@Fedik sad that you think so - at first you denied there was even a problem and it was my faulty for not knowing I should change an undocumented setting to make my site work. Thankfully Harald understood and set this as a release blocker. now you want to stick a band aid on it to fix the problem even though you are not fixing the problem you are just pretending that a setting in the plugin should be ignored if its set to 750 but accepted if its set to 749 or 751.

Fedik · 2026-03-07T09:41:53Z

at first you denied there was even a problem and it was my faulty for not knowing I should change an undocumented setting to make my site work

Where I said it is your problem and not a bug?

now you want to stick a band aid on it to fix the problem even though you are not fixing the problem

It is a proper fix, with fallback to default values. The TinyMCE a bit tricky, but I am fine with that.

If you know how to update plugin parameters during upgrade and want to help then you are free to contribute to this PR. Until then the fix is good.

if its set to 750 but accepted if its set to 749 or 751.

You want to fight for 1px, seriously?

brianteeman · 2026-03-07T11:00:31Z

You completely miss the point but that doesnt suprise me at all.

Plugin paramaters can be updated in a PR - we've done it before just look and you will see, I woujld expect a maintainer to know this

Again you miss the point - this PR replaces 750 woith 100% but if its 749 or 751 then it is left as it is.

plugins/editors/tinymce/src/PluginTraits/DisplayTrait.php

brianteeman · 2026-03-07T12:19:27Z

Dont really want to create a new issue but this is confusing or another error? I can create a new issue if you prefer

PLG_TINY_FIELD_HTMLHEIGHT_DESC="Set the height of the editor. Leave empty for automatic height. A numeric value can be used or a value with valid CSS units (for example: 50vh or 50rem)."
PLG_TINY_FIELD_HTMLWIDTH_DESC="Set the width of the editor. Leave empty for 100% width. A numeric value can be used or a value with valid CSS units (for example: 50vw or 50em)."

You cannot leave it as empty. If you delete the contents of the field then on save the values of 550px and 100% height are inserted

chrome_bvOMUWs7DH.mp4

Fedik · 2026-03-07T12:54:07Z

Yes, the description is misleading, and never worked like that because the form has a default values.
Please open a new issue or PR for the description.

We can make work like that, but then we should remove default values from the from.
In any case that will be a different issue/fix.

tecpromotion · 2026-03-09T16:58:24Z

I have tested this item ✅ successfully on fa546b7

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47305.}

HLeithner · 2026-03-09T17:00:48Z

I have tested this item ✅ successfully on fa546b7

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47305.}

brianteeman · 2026-03-09T17:04:43Z

Created a new issue #47340 as requested for the incorrect language strings ~~introduced in this PR - I dont understand at all why this is being accepted with incorrect language strings. No other PR would be merged with a known issue~~ sorry I now realise the faulty language strings were in the original PR not this fix

HLeithner · 2026-03-09T17:47:53Z

good, now only the language fix is missing for #47340 and I hope than this section is solved after 15 years or so...

richard67 · 2026-03-09T17:54:06Z

good, now only the language fix is missing for #47340 and I hope than this section is solved after 15 years or so...

Don’t forget to close the issue when all is done.

Fix none / codemirror editor width and height

09153a2

joomla-cms-bot added the PR-6.1-dev label Mar 5, 2026

RickR2H mentioned this pull request Mar 5, 2026

[6.1] Fix TinyMCE editor not respecting width and height settings #46438

Merged

4 tasks

brianteeman reviewed Mar 5, 2026

View reviewed changes

libraries/src/Form/Field/EditorField.php Outdated Show resolved Hide resolved

Merge branch '6.1-dev' into fix-none-editor

30f258a

RickR2H added 3 commits March 5, 2026 15:20

Plugin specific implementation of the width and height fallback. Rese…

075f2fa

…t default editor width 750px to 100% for installations upgraded from older versions.

Merge branch 'fix-none-editor' of github.com:RickR2H/joomla-cms into …

4a1d39d

…fix-none-editor

Fix code style

a1b5370

RickR2H mentioned this pull request Mar 5, 2026

Fix: Mail template editor Styling(#47292) #47302

Open

5 tasks

Fedik reviewed Mar 5, 2026

View reviewed changes

plugins/editors/codemirror/layouts/editors/codemirror/codemirror.php Show resolved Hide resolved

RickR2H added 2 commits March 5, 2026 15:42

Fix code indetation

1edd018

Move logic for width and height. Added missing default value for width

e09f5be

HLeithner added bug Release Blocker labels Mar 6, 2026

Fedik linked an issue Mar 6, 2026 that may be closed by this pull request

[6.1] Mail Templates Regression #47292

Closed

brianteeman reviewed Mar 6, 2026

View reviewed changes

plugins/editors/tinymce/src/PluginTraits/DisplayTrait.php Outdated Show resolved Hide resolved

HLeithner mentioned this pull request Mar 7, 2026

Revert (#46438) [6.1] Fix TinyMCE editor not respecting width and height set #47325

Closed

5 tasks

Fedik mentioned this pull request Mar 7, 2026

[6.1] Add update SQL script for PR #46438 to fix TinyMCE editor default width and height settings on core update #47326

Merged

5 tasks

richard67 reviewed Mar 7, 2026

View reviewed changes

plugins/editors/tinymce/src/PluginTraits/DisplayTrait.php Outdated Show resolved Hide resolved

richard67 changed the title ~~Fix none/codemirror editor width and height~~ [6.1] Fix none/codemirror editor width and height Mar 7, 2026

RickR2H and others added 2 commits March 9, 2026 16:24

Removed the reset to 100%

ad8893a

Merge branch '6.1-dev' into fix-none-editor

fa546b7

brianteeman mentioned this pull request Mar 9, 2026

[6.1] tinymce editor and height descriptions #47340

Open

HLeithner merged commit 7908ae9 into joomla:6.1-dev Mar 9, 2026
53 checks passed

HLeithner added this to the Joomla! 6.1.0 milestone Mar 9, 2026

tecpromotion removed the Release Blocker label Mar 9, 2026

Uh oh!

Conversation

RickR2H commented Mar 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary of Changes

Testing Instructions

Additional Testing Instructions

NOTE:

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Link to documentations

Uh oh!

brianteeman commented Mar 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

RickR2H commented Mar 5, 2026

Uh oh!

brianteeman commented Mar 5, 2026

Uh oh!

Uh oh!

Uh oh!

brianteeman commented Mar 6, 2026

Uh oh!

brianteeman commented Mar 6, 2026

Uh oh!

RickR2H commented Mar 7, 2026

Uh oh!

brianteeman commented Mar 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Fedik commented Mar 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

brianteeman commented Mar 7, 2026

Uh oh!

Fedik commented Mar 7, 2026

Uh oh!

brianteeman commented Mar 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

brianteeman commented Mar 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Fedik commented Mar 7, 2026

Uh oh!

tecpromotion commented Mar 9, 2026

Uh oh!

HLeithner commented Mar 9, 2026

Uh oh!

brianteeman commented Mar 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

HLeithner commented Mar 9, 2026

Uh oh!

richard67 commented Mar 9, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

RickR2H commented Mar 5, 2026 •

edited

Loading

brianteeman commented Mar 5, 2026 •

edited

Loading

brianteeman commented Mar 7, 2026 •

edited

Loading

Fedik commented Mar 7, 2026 •

edited

Loading

brianteeman commented Mar 7, 2026 •

edited

Loading

brianteeman commented Mar 7, 2026 •

edited

Loading

brianteeman commented Mar 9, 2026 •

edited

Loading